Skip to content

Register browser even if get url fails. - #263

Merged
ombre42 merged 4 commits into
robotframework:masterfrom
emanlove:master
Jan 24, 2014
Merged

ombre42 merged 4 commits into
robotframework:masterfrom
emanlove:master

Conversation

@emanlove

@emanlove emanlove commented Jan 9, 2014

Copy link
Copy Markdown
Member

Test and patch for fixing issue #70.

An issue was reported with SeleniumLibrary [1] where if "Open Browser"
succeeds in starting the browser but the get url call fails the opened
browser doesn't get registered with the Library's internal browser registery.
Thus when 'Close All Browsers' is called it fails to close the opened browser.
This patch fixes that problem.

[1] http://code.google.com/p/robotframework-seleniumlibrary/issues/detail?id=220
@ombre42

ombre42 commented Jan 24, 2014

Copy link
Copy Markdown
Contributor

Looks good to me

ombre42 added a commit that referenced this pull request Jan 24, 2014
Register browser even if get url fails.
@ombre42
ombre42 merged commit bad7fa8 into robotframework:master Jan 24, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants